Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Authorisation adjustment sample update #96

Merged
merged 12 commits into from
Aug 28, 2023
Merged

Authorisation adjustment sample update #96

merged 12 commits into from
Aug 28, 2023

Conversation

Kwok-he-Chu
Copy link
Contributor

Changes

  • PaymentModel.cs stores properties of the latest payment (which is shown in the admin panel)
  • PaymentModel.cs contains a List (payment details history) which are populated when
    • An preauthorisation payment request is made
    • A new webhook is received with updates on the preauthorisation (e.g. adjusted/captured/reversal etc.)
  • PaymentModel contains a PaymentStatus that is calculated based on the payment details history above

Minor changes/clean-up.

  • Renamed Reference to MerchantReference for consistency
  • Updated comments to include a summary of each property.
  • Updated admin panel view and CSS
  • Change mymusic/myhotel logo and image names
  • Use active-form (verbs) in the Admin Panel
  • Remove "type" from the javascript file and the cshtml, which was unused since we're only supporting the type == "card"

Kwok-he-Chu and others added 7 commits August 24, 2023 16:36
@Kwok-he-Chu
Update Authorisation Adjustment Example
5f83263 
Rename Reference to MerchantReference for consistency

Separate the logic in the repository for inserting the PaymentModel and the PaymentDetailsModel (history)

Updated comments to include a summary of each property.

Updated admin panel view and CSS

Change mymusic/myhotel logo and image names
Use active-form (verbs) in the Admin Panel
Remove "type" from the javascript file and the cshtml, which was unused since only "card" is supported
@Kwok-he-Chu
WIP
6e31b36
@Kwok-he-Chu
Added current PaymentStatus
1b5bf3d
@Kwok-he-Chu
Using fix
6490d80
@Kwok-he-Chu
Take CAPTURE_FAILED, REFUNDED_REVERSED and REFUND_FAILED into account
c5a263e
@Kwok-he-Chu
Add logic in webhook controller to update the amount to the latest au…
9650103 
…thorisation adjustment amount
Renames
7cad472
@Kwok-he-Chu Kwok-he-Chu merged commit 14139ef into main Aug 28, 2023
2 of 7 checks passed
@Kwok-he-Chu Kwok-he-Chu deleted the authorisation-adjustment-sample-update branch August 28, 2023 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gcatanese gcatanese gcatanese approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Kwok-he-Chu @gcatanese